Skip to content

Conversation

@jacek-poreda-sonarsource
Copy link
Contributor

Accidentally closed PR, so here is reference to it: #92

@jacek-poreda-sonarsource
Copy link
Contributor Author

@dbmeneses You can have a look again, I think we have addressed all things you have mentioned

@sonarsource-next
Copy link

SonarQube Quality Gate

Quality Gate failed

Failed condition 16.3% 16.35% Duplicated Lines (%) on New Code (is greater than 3%)

See analysis details on SonarQube

@alain-kermis-sonarsource
Copy link
Contributor

@dbmeneses: would you mind having a look at the changes I made to changelog in this commit. Many thanks!

@dbmeneses
Copy link
Collaborator

@dbmeneses: would you mind having a look at the changes I made to changelog in this commit. Many thanks!

Looks good to me, thanks!

- Use new types for identity providers, deprecate javax methods
- Introduce HttpFilter class and deprecate ServletFilter which is using javax.servlet.*
- Add new methods required by filters in SonarQube
- Deprecate javax.servlet usage in org.sonar.api.security package

Co-authored-by: Eric Giffon <[email protected]>
Co-authored-by: Alain Kermis <[email protected]>
Co-authored-by: Antoine Vinot <[email protected]>
Co-authored-by: Jacek Poreda <[email protected]>
@jacek-poreda-sonarsource jacek-poreda-sonarsource force-pushed the task/PLUGINAPI-42-replace-javax branch from f9a55eb to dcf517d Compare April 21, 2023 07:10
@sonarsource-next
Copy link

SonarQube Quality Gate

Quality Gate failed

Failed condition 16.3% 16.35% Duplicated Lines (%) on New Code (is greater than 3%)

See analysis details on SonarQube

@jacek-poreda-sonarsource jacek-poreda-sonarsource merged commit 42ca0a6 into master Apr 21, 2023
@jacek-poreda-sonarsource jacek-poreda-sonarsource deleted the task/PLUGINAPI-42-replace-javax branch April 21, 2023 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants